Skip to content

feat: General cleanup of legacy Studio page waffle flags and related code#38424

Merged
irfanuddinahmad merged 1 commit into
masterfrom
iahmad/DEPR-36108-general-cleanup
May 13, 2026
Merged

feat: General cleanup of legacy Studio page waffle flags and related code#38424
irfanuddinahmad merged 1 commit into
masterfrom
iahmad/DEPR-36108-general-cleanup

Conversation

@irfanuddinahmad
Copy link
Copy Markdown
Contributor

@irfanuddinahmad irfanuddinahmad commented Apr 23, 2026

This pull request primarily refactors utility functions in cms/djangoapps/contentstore/utils.py to simplify and standardize how URLs for various course authoring microfrontend pages are constructed and returned. Additionally, it includes some minor cleanup in the toggles file by removing unnecessary blank lines.

Refactoring and simplification of URL utility functions:

  • Standardized the pattern for generating and returning URLs in several functions (such as get_schedule_details_url, get_advanced_settings_url, get_grading_url, get_course_team_url, get_import_url, get_export_url, get_certificates_url, and get_group_configurations_url) by assigning the constructed URL to a variable and returning it if the base URL exists, otherwise returning None. This improves readability and consistency across these functions. [1] [2] [3] [4]

Code cleanup:

  • Removed extra blank lines in cms/djangoapps/contentstore/toggles.py to tidy up the code. [1] [2]

@irfanuddinahmad irfanuddinahmad marked this pull request as draft April 23, 2026 08:42
@irfanuddinahmad irfanuddinahmad force-pushed the iahmad/DEPR-36108-general-cleanup branch from a7f131e to cdf4005 Compare April 28, 2026 11:36
@irfanuddinahmad irfanuddinahmad linked an issue Apr 28, 2026 that may be closed by this pull request
52 tasks
@irfanuddinahmad irfanuddinahmad force-pushed the iahmad/DEPR-36108-general-cleanup branch 6 times, most recently from 4fa8991 to 2514cb6 Compare May 6, 2026 10:06
@irfanuddinahmad irfanuddinahmad force-pushed the iahmad/DEPR-36108-general-cleanup branch 10 times, most recently from f89158d to 5031a59 Compare May 12, 2026 15:29
@irfanuddinahmad irfanuddinahmad marked this pull request as ready for review May 12, 2026 16:12
… code.

Remove the waffle flags and helper functions for all legacy Studio pages
that have been replaced by the Authoring MFE:
- legacy_studio.schedule_details
- legacy_studio.advanced_settings
- legacy_studio.grading
- legacy_studio.import
- legacy_studio.export
- legacy_studio.course_team
- legacy_studio.certificates
- legacy_studio.configurations

Also removes the course_waffle_flags REST API endpoint
(GET /api/contentstore/v1/course_waffle_flags) and its serializer, view,
and tests, as these only existed to expose the now-removed flags.

Simplifies URL helpers in utils.py to unconditionally return MFE URLs.

This work is part of #36108

BREAKING CHANGE: All of the above waffle flags are removed. The
GET /api/contentstore/v1/course_waffle_flags endpoint is removed.
@irfanuddinahmad irfanuddinahmad force-pushed the iahmad/DEPR-36108-general-cleanup branch from 5031a59 to bc589aa Compare May 13, 2026 02:12
@irfanuddinahmad irfanuddinahmad enabled auto-merge May 13, 2026 05:11
@irfanuddinahmad irfanuddinahmad merged commit d5111c7 into master May 13, 2026
46 checks passed
@irfanuddinahmad irfanuddinahmad deleted the iahmad/DEPR-36108-general-cleanup branch May 13, 2026 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DEPR]: Various legacy Studio pages with direct MFE replacements

2 participants